Skip to content

fix(cli): improve error unwrapping and accept degraded health status#5

Merged
chrlesur merged 1 commit intoCloud-Temple:mainfrom
BeArchiTek:fix/cli-error-handling-and-health-status
Apr 22, 2026
Merged

fix(cli): improve error unwrapping and accept degraded health status#5
chrlesur merged 1 commit intoCloud-Temple:mainfrom
BeArchiTek:fix/cli-error-handling-and-health-status

Conversation

@BeArchiTek
Copy link
Copy Markdown
Contributor

@BeArchiTek BeArchiTek commented Apr 13, 2026

Summary

  • Unwrap ExceptionGroup/TaskGroup errors in MCPClient.call_tool to surface the real underlying error (e.g. HTTP 401) instead of showing the opaque ExceptionGroup wrapper. The MCP SDK uses anyio TaskGroups, which wrap failures in BaseExceptionGroup.
  • Add degraded to accepted health-check statuses in the CLI so that partial-health responses are displayed correctly instead of being treated as errors.

Test plan

  • Trigger an MCP auth failure (e.g. invalid token) and verify the CLI now shows the actual HTTP error message
  • Run health command against a server returning degraded status and verify it displays normally

- Unwrap ExceptionGroup/TaskGroup to surface the real MCP error
  (e.g. HTTP 401) instead of showing the opaque wrapper
- Add "degraded" to the list of accepted health-check statuses
  so partial-health responses are displayed correctly
@chrlesur chrlesur merged commit cc3083d into Cloud-Temple:main Apr 22, 2026
chrlesur added a commit that referenced this pull request Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants